Skip to content

update paper link#1128

Merged
Kipok merged 2 commits intomainfrom
wedu/update-nemotron-math-paper
Dec 18, 2025
Merged

update paper link#1128
Kipok merged 2 commits intomainfrom
wedu/update-nemotron-math-paper

Conversation

@wedu-nvidia
Copy link
Collaborator

@wedu-nvidia wedu-nvidia commented Dec 18, 2025

Summary by CodeRabbit

  • Documentation
    • Updated Nemotron-Math paper references to point to the arXiv abstract page instead of local PDF files, maintaining publication information and display details.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Wei Du <wedu@nvidia.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 18, 2025

📝 Walkthrough

Walkthrough

Updated Nemotron-Math paper references from local PDF paths to the arXiv abstract URL (https://arxiv.org/abs/2512.15489) in two documentation files. Links modified in both the main releases index and the Nemotron-Math v2 release notes.

Changes

Cohort / File(s) Summary
Documentation link updates
docs/releases/index.md, docs/releases/nemotron-math-v2/index.md
Replaced local paper PDF reference (./nemotron-math-v2/paper.pdf) with arXiv abstract URL (https://arxiv.org/abs/2512.15489); two occurrences updated across both files

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • gwarmstrong
  • Kipok

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'update paper link' is vague and generic. While it refers to the actual change in the PR (updating paper links), it lacks specificity about which paper or the nature of the update. Consider using a more specific title such as 'Update Nemotron-Math paper link to arXiv' to clearly indicate which paper is being updated and why.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch wedu/update-nemotron-math-paper

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/releases/nemotron-math-v2/index.md (1)

23-23: Add {:target="_blank"} for consistency with other paper links.

Line 23 updates the paper link to the arXiv URL, but it lacks the {:target="_blank"} attribute that all other paper links in docs/releases/index.md include. This ensures external links open in a new tab, providing consistent UX across the documentation.

🔎 Apply this diff to add the target attribute:
-See our [paper](https://arxiv.org/abs/2512.15489) to learn more details!
+See our [paper](https://arxiv.org/abs/2512.15489){:target="_blank"} to learn more details!
📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1c0722a and 63e8730.

📒 Files selected for processing (2)
  • docs/releases/index.md (1 hunks)
  • docs/releases/nemotron-math-v2/index.md (1 hunks)
🔇 Additional comments (1)
docs/releases/index.md (1)

25-25: ✅ Link update looks good.

The paper reference has been correctly updated from a local path to the arXiv abstract URL, with proper {:target="_blank"} attribute for external link handling. The change aligns with the project's documentation standards.

@Kipok Kipok enabled auto-merge (squash) December 18, 2025 03:21
@Kipok Kipok merged commit 1cac9db into main Dec 18, 2025
5 checks passed
@Kipok Kipok deleted the wedu/update-nemotron-math-paper branch December 18, 2025 03:37
wasiahmad pushed a commit that referenced this pull request Dec 19, 2025
Signed-off-by: Wei Du <wedu@nvidia.com>
wasiahmad pushed a commit that referenced this pull request Dec 19, 2025
Signed-off-by: Wei Du <wedu@nvidia.com>

Signed-off-by: wasiahmad <wasiahmad@ucla.edu>
blahblahasdf pushed a commit to blahblahasdf/Skills that referenced this pull request Jan 8, 2026
Signed-off-by: Wei Du <wedu@nvidia.com>
Signed-off-by: dlord <dlord@nvidia.com>
hsiehjackson pushed a commit that referenced this pull request Jan 13, 2026
Signed-off-by: Wei Du <wedu@nvidia.com>
Signed-off-by: Cheng-Ping Hsieh <chsieh@nvidia.com>
wasiahmad pushed a commit that referenced this pull request Feb 4, 2026
Signed-off-by: Wei Du <wedu@nvidia.com>
dgtm777 pushed a commit that referenced this pull request Mar 18, 2026
Signed-off-by: Wei Du <wedu@nvidia.com>
dgtm777 pushed a commit that referenced this pull request Mar 18, 2026
Signed-off-by: Wei Du <wedu@nvidia.com>
Signed-off-by: dgitman <dgitman@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants